Skip to content

Conversation

@dkalinowski
Copy link
Collaborator

@dkalinowski dkalinowski commented Oct 30, 2025

🛠 Summary

CVS-170537

  • added support for comma separated bind address list via CLI/C-API
  • changed localhost to 127.0.0.1 in docs where requests pip package is used because request localhost can introduce initial 2s delay on windows systems due to ipv6 connection try before actual ipv4 connection is estabilished
  • updated security considerations

@dkalinowski dkalinowski changed the title Allow more than one http rest listeners Allow more than one http rest & grc listeners Nov 5, 2025
@dkalinowski dkalinowski changed the title Allow more than one http rest & grc listeners Allow more than one http rest & grpc listeners Nov 5, 2025

## Network Configuration for Optimal Performance

When clients connect to the server using hostname resolution (particularly "localhost"), the system may attempt IPv6 resolution first before falling back to IPv4. If IPv6 is disabled, misconfigured, or unavailable, this can cause connection timeouts and delays before the IPv4 fallback occurs, which is especially noticeable when minimizing time to first token in generative AI applications.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By default, OVMS endpoints are bound to all ipv4 addresses. On same systems, which route localhost name to ipv6 address, it might cause extra time on the client side to switch to ipv4. It can effectively results with extra 1-2s latency.
It can be overcome by switching the API URL to http://127.0.0.1 instead.

Alternatively ipv6 can be enabled in the model server using --grpc_bind_address and --rest_bind_address.
For example:
--grpc_bind_address 127.0.0.1,::1 --rest_bind_address 127.0.0.1,::1
or
--grpc_bind_address 0.0.0.0,:: --rest_bind_address 0.0.0.0,::

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed

@dtrawins dtrawins added this to the 2025.4rc milestone Nov 7, 2025
@dkalinowski dkalinowski merged commit b80c17d into main Nov 12, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants